Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: implement a textual summary for DWI #1291

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

celprov
Copy link
Collaborator

@celprov celprov commented Apr 23, 2024

enh: implement an interface to generate a textual summary for DWI
enh: save the summary in html form using DerivativesDataSink
enh: modify models output of NumberOfShells so it returns 0 in case data is not DSI

Two problems to solve:

  1. The PhaseEncodingDirection is not captured by inheritance by BIDSLayout. Maybe it is related to this issue
  2. The summary.html gets correctly saved under sub-001/figures/ but do not get incorporated into the report

celprov and others added 5 commits April 17, 2024 15:54
enh: save the summary in html form using DerivativesDataSink
enh: modify models output of NumberOfShells so it returns 0 in case data is not DSI
fix: model type was mistakenly reversed
fix: import DerivativesDataSink from mriqc.interfaces rather than niworkflows
fix: bootstrap dwi entities to find summary.html
fix: keep only basename for filename
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant